Skip to content

Switch Network Exporter image to be generic#1548

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
karelyatin:OSPRH-18529
Jul 31, 2025
Merged

Switch Network Exporter image to be generic#1548
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
karelyatin:OSPRH-18529

Conversation

@karelyatin

@karelyatin karelyatin commented Jul 29, 2025 •
edited by openshift-ci Bot
Loading

Copy link
Copy Markdown
Contributor

Network Exporter being extended to be used on Control plane pods, so make it generic.

Resolves: #OSPRH-18529

@openshift-ci openshift-ci Bot requested review from abays and dprince July 29, 2025 10:46
@softwarefactory-project-zuul

softwarefactory-project-zuul Bot commented Jul 29, 2025

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/cbaeba46529a42fe8ebec8ac5b8e4120

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 55m 23s
❌ podified-multinode-edpm-deployment-crc FAILURE in 1h 36m 17s
❌ cifmw-crc-podified-edpm-baremetal RETRY_LIMIT in 19m 15s
❌ adoption-standalone-to-crc-ceph-provider FAILURE in 2h 41m 38s
❌ openstack-operator-tempest-multinode FAILURE in 1h 39m 52s

@karelyatin

karelyatin commented Jul 30, 2025

Copy link
Copy Markdown
Contributor Author

/retest

@karelyatin

karelyatin commented Jul 30, 2025

Copy link
Copy Markdown
Contributor Author

/test functional unrelated [FAIL] OpenStackOperator controller when A OpenStackControlplane instance is created [It] should have configured nova from the service template

@karelyatin karelyatin requested a review from stuggi July 30, 2025 08:44
stuggi
stuggi reviewed Jul 30, 2025
View reviewed changes
Comment thread pkg/dataplane/inventory.go Outdated
@karelyatin
Switch Network Exporter image to be generic
e31b19f 
Network Exporter being extended to be used on Control
plane pods, so make it generic.

Resolves: #OSPRH-18529
@karelyatin

karelyatin commented Jul 30, 2025

Copy link
Copy Markdown
Contributor Author

/retest cluster wait

@openshift-ci

openshift-ci Bot commented Jul 30, 2025

Copy link
Copy Markdown
Contributor

@karelyatin: The /retest command does not accept any targets.
The following commands are available to trigger required jobs:

/test functional

/test images

/test openstack-operator-build-deploy

/test openstack-operator-build-deploy-kuttl

/test openstack-operator-build-deploy-kuttl-4-18

/test precommit-check

The following commands are available to trigger optional jobs:

/test openstack-operator-build-deploy-tempest

Use /test all to run all jobs.

Details

In response to this:

/retest cluster wait

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@karelyatin

karelyatin commented Jul 30, 2025

Copy link
Copy Markdown
Contributor Author

/retest

stuggi
stuggi approved these changes Jul 31, 2025
View reviewed changes

@stuggi stuggi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci openshift-ci Bot added the lgtm label Jul 31, 2025
@openshift-ci

openshift-ci Bot commented Jul 31, 2025

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: karelyatin, stuggi

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@karelyatin

karelyatin commented Jul 31, 2025

Copy link
Copy Markdown
Contributor Author

/retest

@openshift-merge-bot openshift-merge-bot Bot merged commit eab3fc3 into openstack-k8s-operators:main Jul 31, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@stuggi stuggi stuggi approved these changes

@abays abays Awaiting requested review from abays

@dprince dprince Awaiting requested review from dprince

Assignees

@stuggi stuggi

Labels

approved lgtm

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@karelyatin @stuggi